Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat: credential status notifications #212

Merged
merged 45 commits into from
Aug 2, 2023
Merged

feat: credential status notifications #212

merged 45 commits into from
Aug 2, 2023

Conversation

sjoerdlmkns
Copy link
Contributor

No description provided.

@sjoerdlmkns
chore: provide bloc
06151d7
@sjoerdlmkns
wip: NotificationsScreen, NotificationsCard, NotificationsBell
fc150a1
@sjoerdlmkns
feat: empty notifications indicator
1f29cd5
@sjoerdlmkns
feat: IrmaDismissible
d461e23
@sjoerdlmkns
feat: added timestamp to notification
05ccd39
@sjoerdlmkns
feat: build notifications according to bloc
7f0a858
@sjoerdlmkns
feat: add DeleteNotificationEvent on notification dismiss
d5301a6
@sjoerdlmkns
test: reach NotificationsScreen
b61c762
@sjoerdlmkns
test: empty notifications test
f509fd3
@sjoerdlmkns
fix: timestamp json serialize
5a3aac9
@sjoerdlmkns
refactor: removed serializedCredentialStatusNotifications pref
254146f
@sjoerdlmkns
feat: reset notifications on corrupted cache
3934da2
@sjoerdlmkns
test: load valid/corrupted cache
fe2abbd
@sjoerdlmkns
fix: check empty string before serializing
7086c62
@sjoerdlmkns
test: notificaitons filled-state
59717bb
@sjoerdlmkns
Merge branch 'master' into notifications-ui
98d3dab
@sjoerdlmkns
feat: add read to notification
b34e480
@sjoerdlmkns
test: mark-notifications-as-read
1bc66a8
@sjoerdlmkns
feat: mark notifications as read when leaving NotificationsScreen
9cd01ea
@sjoerdlmkns
test: read-all-notifications
081f496
@sjoerdlmkns
fix: serialize notification action
d832683
@sjoerdlmkns
fix: wrong json type key
81eadc3
@sjoerdlmkns
test: reuse mocked serialized notifications
74fdcb6
@sjoerdlmkns
feat: handle CredentialDetailNavigationAction on NotificaitonsScreen
583de3d
@sjoerdlmkns
test: also reuse mocked serialized notifications in integration tests
e8109ed
@sjoerdlmkns
test: dismiss notification integration test
5d2e7e7
@sjoerdlmkns
chore: updated CHANGELOG
ceba972
@sjoerdlmkns
feat: mark single notifiation as read on action
6679312
@sjoerdlmkns
test: mark single notfication as read
1e9bb2a
@sjoerdlmkns
test: credential status notification action
7b399db
@sjoerdlmkns sjoerdlmkns marked this pull request as ready for review July 27, 2023 07:31
sanderhollaar
sanderhollaar previously approved these changes Aug 1, 2023
View reviewed changes
@sjoerdlmkns sjoerdlmkns requested a review from ivard August 1, 2023 12:11
@sjoerdlmkns sjoerdlmkns merged commit 4e40ff1 into master Aug 2, 2023
14 checks passed
@sjoerdlmkns sjoerdlmkns deleted the notifications-ui branch August 2, 2023 09:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ivard ivard ivard approved these changes

@sanderhollaar sanderhollaar sanderhollaar approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sjoerdlmkns @ivard @sanderhollaar